Skip to content

Use new GH action to upload SDK version info#2222

Merged
fnewberg merged 3 commits intomainfrom
fnewberg/use-upload-sdk-version-action
Jun 2, 2025
Merged

Use new GH action to upload SDK version info#2222
fnewberg merged 3 commits intomainfrom
fnewberg/use-upload-sdk-version-action

Conversation

@fnewberg
Copy link
Copy Markdown
Contributor

Goal

  • upload SDK version release data to our version metadata store

Testing

Release Notes

WHAT:

WHY:

@fnewberg fnewberg requested a review from vitaliyf May 30, 2025 21:56
@fnewberg fnewberg requested a review from a team as a code owner May 30, 2025 21:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 30, 2025

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

.github/workflows/release-workflow.yml

PackageVersionLicenseIssue Type
embrace-io/public-actions/upload-sdk-versionf4229398f257b24dbbe8b873f78e719f9af6cbbbNullUnknown License

OpenSSF Scorecard

PackageVersionScoreDetails
actions/embrace-io/public-actions/upload-sdk-version f4229398f257b24dbbe8b873f78e719f9af6cbbb UnknownUnknown

Scanned Files

  • .github/workflows/release-workflow.yml

@codecov
Copy link
Copy Markdown

codecov bot commented May 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.62%. Comparing base (f4cb1a8) to head (fd4c3b6).
Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2222   +/-   ##
=======================================
  Coverage   85.62%   85.62%           
=======================================
  Files         465      465           
  Lines       10379    10379           
  Branches     1522     1522           
=======================================
  Hits         8887     8887           
  Misses        835      835           
  Partials      657      657           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

timeout-minutes: 60
steps:
- name: Validate access to version data service
uses: embrace-io/public-actions/upload-sdk-version@f4229398f257b24dbbe8b873f78e719f9af6cbbb
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this pinned ID for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's the sha for the version of the upload-sdk-version action. GH warns you if you do anything less strict than that for 3rd party actions. this may seem like a bit of overkill, but is probably a response to attacks that have occurred in the past where people changed the behavior of public actions. even though we control the action, GH sees it as a 3rd party one

@fnewberg fnewberg merged commit 4e3fad7 into main Jun 2, 2025
11 checks passed
@fnewberg fnewberg deleted the fnewberg/use-upload-sdk-version-action branch June 2, 2025 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants